Skip to content
This repository was archived by the owner on Jan 20, 2021. It is now read-only.

vm: backend migration changes support#789

Open
shwstppr wants to merge 12 commits into
apache:masterfrom
shapeblue:vm-migrate-backend-improvements
Open

vm: backend migration changes support#789
shwstppr wants to merge 12 commits into
apache:masterfrom
shapeblue:vm-migrate-backend-improvements

Conversation

@shwstppr

@shwstppr shwstppr commented Oct 6, 2020

Copy link
Copy Markdown
Contributor

Backend changes at apache/cloudstack#4385

Use migrateVirtualMachineWithVolume for stopped VM migration to a different storage pool
VR and system VM migration between cluster changes

Screenshots:
Screenshot from 2020-11-13 11-36-59

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@blueorangutan

Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@yadvr yadvr added this to the 1.1 milestone Oct 6, 2020
@borisstoyanov

Copy link
Copy Markdown

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@borisstoyanov a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/789 (JID-3641)

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

@yadvr yadvr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, needs manual testing against backend PR

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr

Copy link
Copy Markdown
Contributor Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@shwstppr shwstppr requested a review from yadvr November 20, 2020 11:45
@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/789 (JID-3682)

@shwstppr

Copy link
Copy Markdown
Contributor Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/789 (JID-3731)

@shwstppr shwstppr marked this pull request as ready for review January 18, 2021 07:36
@shwstppr

Copy link
Copy Markdown
Contributor Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/789 (JID-3814)

@yadvr

yadvr commented Jan 18, 2021

Copy link
Copy Markdown
Member

@shwstppr can you add the link to the backend PR on PR description

@shwstppr

shwstppr commented Jan 18, 2021

Copy link
Copy Markdown
Contributor Author

@rhtyd it was there in the end. Moved it to top

Comment thread src/locales/en.json Outdated
"label.storage.tags": "Storage Tags",
"label.storage.traffic": "Storage Traffic",
"label.storageid": "Primary Storage",
"label.storagemigrationrequired": "Storage Migration Required",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"label.storage.migration.required" key might look better

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants